Skip to content

Broadcasts: support customScoring and multi time-control stats#2718

Merged
veloce merged 16 commits intolichess-org:mainfrom
allanjoseph98:broadcast-api-changes
Mar 9, 2026
Merged

Broadcasts: support customScoring and multi time-control stats#2718
veloce merged 16 commits intolichess-org:mainfrom
allanjoseph98:broadcast-api-changes

Conversation

@allanjoseph98
Copy link
Copy Markdown
Member

Simulator.Screen.Recording.-.iPhone.17.-.2026-03-07.at.17.24.51.mov
  • Support multi time-control broadcast stats. Affects the
    i. Players screen
    ii. Player's games screen
Simulator.Screen.Recording.-.iPhone.17.-.2026-03-07.at.17.25.48.mov

@ijm8710
Copy link
Copy Markdown

ijm8710 commented Mar 7, 2026

Hi @allanjoseph98

This is very cool, I had a few open tickets for some of the things in your first video so I'm happy to see them.

Curious to grab your thoughts on 2 things

  • I love how you colored wins/losses on the player results screen (consistent with web). The scenario where you have a red rating change while the tie is non-colored can cause some imbalance/noise.
    Personally I think it makes sense to move the rating change to the left side after the rating itself rather than have it clashing with the two right next to each other.
    What are your thoughts?

    • benefit 1 is it allows the user to see the win/loss color with less noise
    • Benefit 2 is it becomes similar to the "standings" section where the rating change is right after the rating. Plus that just makes sense for it to be naturally anyway
  • Not everyone has agreed with me but I really like how the player results screen on web is numbered down. It makes it much easier to follow an event with a bunch of rounds in a row so you know where you last left off. And this is how it is on web too. Thoughts?

image
image

@allanjoseph98
Copy link
Copy Markdown
Member Author

  1. The rating change on the results screen is for the player you're viewing not for the opponent in the list. You can't move one player's rating diff next to their opponent's rating.
  2. This is a separate feature request that you should file an issue for.

@ijm8710
Copy link
Copy Markdown

ijm8710 commented Mar 8, 2026

  1. The rating change on the results screen is for the player you're viewing not for the opponent in the list. You can't move one player's rating diff next to their opponent's rating.

I meant move the rating diff next to the rating itself on the players games list. This will mimic how it is for standings. And by doing so, it would allow the win (green) loss (red) tie (no color) to be on right side alone without the distraction of the rating change while allowing the rating change to be next to the rating itself

image
image

@allanjoseph98
Copy link
Copy Markdown
Member Author

I understood you. You didn't understand me. Those points are what Gergely gained for beating Marcell. Moving Gergely's rating gain for that game next to Marcell's rating reads like Marcell gained 3 rating points for that game even though he lost. Suffer through this hideous diagram for your understanding.
559863490-2dc799da-54f8-4f48-8aeb-c0d7c60d01e6~2.jpeg

@ijm8710
Copy link
Copy Markdown

ijm8710 commented Mar 8, 2026

Ah apologies and thanks for taking the time to break that down. Now I hear what you're saying

I still think it gets a little combative where the color from the rating change and the color from the result are stacked in the exact same column

I'm not sure the best way around that whether perhaps putting it next to the name or even to the right in another column like on web.

I'm assuming most feel it's fine as is

I hear you on what you're saying now but seeing two colors at once in the same place that won't always be the same to me can feel a little misdirection

@ijm8710
Copy link
Copy Markdown

ijm8710 commented Mar 8, 2026

.. Or even part of me feels then maybe to remove the color for the rating diff. To me the result color trumps the importance of the rating diff color

Copy link
Copy Markdown
Contributor

@veloce veloce left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Thanks.

@veloce veloce merged commit f87e408 into lichess-org:main Mar 9, 2026
1 check passed
@allanjoseph98 allanjoseph98 deleted the broadcast-api-changes branch March 19, 2026 03:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Broadcast Update: Scores can now be customized

3 participants